Skip to content

fix integration test failure slack message#517

Merged
karawoo merged 1 commit intomainfrom
fix-message
Mar 10, 2026
Merged

fix integration test failure slack message#517
karawoo merged 1 commit intomainfrom
fix-message

Conversation

@karawoo
Copy link
Collaborator

@karawoo karawoo commented Mar 9, 2026

The top-level text at line 73 is apparently only a fallback. I was expecting it to appear in the message but it doesn't; this PR adds the appropriate text in the body of the message.

Checklist

  • Does this change update NEWS.md (referencing the connected issue if necessary)?
  • Does this change need documentation? Have you run devtools::document()?
  • Does this change resolve integration test failures for a preview build of Connect? If so (and if you are on the Connect team), please be sure to update the Connect changelog to reflect that users will need to upgrade their version of connectapi.

Copy link
Collaborator

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this!

@karawoo karawoo merged commit 6f82071 into main Mar 10, 2026
23 checks passed
@karawoo karawoo deleted the fix-message branch March 10, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants